-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bump FRR to 9.1.0 #163
Bump FRR to 9.1.0 #163
Conversation
a8df13d
to
b336198
Compare
any of this references to older frr image should be updated? I suppose the RELEASE note entry is added when releasing. |
Why you do not bump in the latest frr?
if there is a reason could we add it in the description of the commit? |
Good catch! |
I tried the latest, CI started failing for no clear reasons, so I assume latest is not super stable and went to the previous generation which still fixes my problem with importing vrfs. |
Ack, then maybe you could state that not bumping to latest due to ci is failing), and what is the reason you bump (fixes my problem, which problem?) Also we could have an issue "bump to FRR 10" to follow it |
Bumping FRR to the 9.1.0 version. Among others, this fixes importing VRFs when a network is defined in a router belonging to a vrf and another router imports that VRF. Signed-off-by: Federico Paolinelli <[email protected]>
I won't state why I am not bumping on a later version as it might be obsoleted / fixed when we look at it in the future. |
Filed #165 , but not sure I want us to work / bump on that any time soon. |
Is this a BUG FIX or a FEATURE ?:
What this PR does / why we need it:
Bumping FRR to 9.1.0
Special notes for your reviewer:
Release note: